Skip to content

[SPARK-11710] Document new memory management model#9676

Closed
andrewor14 wants to merge 3 commits intoapache:masterfrom
andrewor14:memory-management-docs
Closed

[SPARK-11710] Document new memory management model#9676
andrewor14 wants to merge 3 commits intoapache:masterfrom
andrewor14:memory-management-docs

Conversation

@andrewor14
Copy link
Contributor

@andrewor14 andrewor14 commented Nov 12, 2015

No description provided.

Add a section on memory management overview
812f506 
also remove references to legacy mode configs and update the
description of the new configs to link to this section.
@SparkQA
Copy link

SparkQA commented Nov 12, 2015

Test build #45766 has finished for PR 9676 at commit 812f506.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

rxin
rxin reviewed Nov 13, 2015
View reviewed changes
docs/tuning.md Outdated
Copy link
Contributor

@rxin rxin Nov 13, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i dont' think you'd want to reference a jira ticket, since it is mutable here ... i'd just remove this line.

@andrewor14
Copy link
Contributor Author

andrewor14 commented Nov 13, 2015

@prakashc @vidaha Is it clearer now? Any other feedback?

@prakashc
Copy link

prakashc commented Nov 13, 2015

Looks good.

On Fri, Nov 13, 2015 at 1:23 PM, andrewor14 notifications@github.com
wrote:

@prakashc https://github.com/prakashc @vidaha
https://github.com/vidaha Is it clearer now? Any other feedback?

—
Reply to this email directly or view it on GitHub
#9676 (comment).

@SparkQA
Copy link

SparkQA commented Nov 13, 2015

Test build #45896 has finished for PR 9676 at commit 8afd5bd.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Nov 16, 2015

Test build #46018 has finished for PR 9676 at commit 6838ba6.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@andrewor14
Copy link
Contributor Author

andrewor14 commented Nov 17, 2015

Alright, I'm going to merge this into master and 1.6. Thanks everyone for your feedback.

@asfgit asfgit closed this in 33a0ec9 Nov 17, 2015
asfgit pushed a commit that referenced this pull request Nov 17, 2015
[SPARK-11710] Document new memory management model
32a69e4 
Author: Andrew Or <andrew@databricks.com>

Closes #9676 from andrewor14/memory-management-docs.

(cherry picked from commit 33a0ec9)
Signed-off-by: Andrew Or <andrew@databricks.com>
@andrewor14 andrewor14 deleted the memory-management-docs branch November 17, 2015 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@andrewor14 @SparkQA @prakashc @rxin @vidaha @mrchristine